Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade vitepress to 1.0.0-rc.12 #1844

Merged
merged 7 commits into from
Sep 11, 2023
Merged

upgrade vitepress to 1.0.0-rc.12 #1844

merged 7 commits into from
Sep 11, 2023

Conversation

Fil
Copy link
Contributor

@Fil Fil commented Aug 31, 2023

(We don't have to merge this, but I want to track the changes so we can be ready when v1.0.0 is released.)

Current release: https://github.com/vuejs/vitepress/releases/tag/v1.0.0-rc.12

Summary:

  • light code blocks 👍
  • fork link underline
  • green highlight
  • link underline on the “build on observable” button

Details:

I tried upgrading locally, the nav seems to work.

The new styles for code blocks are quite nicer (the light mode now has light backgrounds).

Capture d’écran 2023-08-31 à 16 41 28

the fork link is now underlined, why not but the underline has two shades, which we'd want to fix either by removing the underline altogether or by having just one shade..

I don't like the dark green they have now put on the selected words in search, the current lighter green is more legible I think.
Capture d’écran 2023-08-31 à 16 40 36

No other issue detected so far.

@Fil Fil requested a review from mbostock August 31, 2023 14:46
@mbostock
Copy link
Member

mbostock commented Sep 8, 2023

Bug for the green: vuejs/vitepress#2902

The default colors in light mode seem strange and not consistent with the dark mode variants.

I computed new purples here: https://observablehq.com/d/4eb7a6328e3dcdb2

@Fil
Copy link
Contributor Author

Fil commented Sep 11, 2023

vuejs/vitepress#2902 was closed with rc.12

@Fil Fil changed the title upgrade vitepress to 1.0.0-rc.10 upgrade vitepress to 1.0.0-rc.12 Sep 11, 2023
@Fil Fil changed the title upgrade vitepress to 1.0.0-rc.12 upgrade vitepress to 1.0.0 Sep 11, 2023
@Fil
Copy link
Contributor Author

Fil commented Sep 11, 2023

With 1.0.0-rc.12 there's an additional underline on the button on the home page.

Capture d’écran 2023-09-11 à 12 08 17

@mbostock mbostock marked this pull request as ready for review September 11, 2023 17:44
@mbostock mbostock changed the title upgrade vitepress to 1.0.0 upgrade vitepress to 1.0.0-rc.12 Sep 11, 2023
@mbostock mbostock enabled auto-merge (squash) September 11, 2023 17:44
@mbostock mbostock merged commit e22c41e into main Sep 11, 2023
1 check passed
@mbostock mbostock deleted the fil/vitepress-1 branch September 11, 2023 17:46
chaichontat pushed a commit to chaichontat/plot that referenced this pull request Jan 14, 2024
* upgrade vitepress to 1.0.0-rc.10

* brand color fixes

* 1.0.0-rc.12

* upgrade dependencies

* recompute brand colors

* fix observable promo button

---------

Co-authored-by: Mike Bostock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants